I looked through this and besides the things I commented on, the things that are there looks good to me. Would love to get webpack 5 merged soon so Trivy can stop shouting about critical security vulnerabilities.

Thanks @pjonsson those were good finds, I have fixed them.

• Looks good! Beside docker comment, just need to add a CHANGES.md line :)

• Do we also need to bump the version in Dockerfile?

@ljowen good catch, and there's a second Dockerfile under deploy/docker. And speaking of simple things that get stuck for a long time in the PR queue, #691 fixes a warning in the Dockerfile.

@na9da does #720 need to be merged before this?

@na9da #726 passed the CI before this change (but after the ESLint update), but I just rebased it and it now fails:

ERROR in ./node_modules/terriajs/test/Models/Catalog/CatalogGroups/OpenDataSoftCatalogGroupSpec.ts:6:23
TS2307: Cannot find module 'fetch-mock' or its corresponding type declarations.
    4 | import OpenDataSoftCatalogItem from "../../../../lib/Models/Catalog/CatalogItems/OpenDataSoftCatalogItem";
    5 | import Terria from "../../../../lib/Models/Terria";
  > 6 | import fetchMock from "fetch-mock";
      |                       ^^^^^^^^^^^^
    7 |
    8 | import facets from "../../../../wwwroot/test/ods/facets.json";
    9 | import datasets from "../../../../wwwroot/test/ods/datasets.json";

The difference is from commit 28aa111 to commit 10c2192.